Do not copy X event in handle_one_xevent except KeyPress case.
authorDmitry Antipov <dmantipov@yandex.ru>
Mon, 16 Sep 2013 10:41:41 +0000 (14:41 +0400)
committerDmitry Antipov <dmantipov@yandex.ru>
Mon, 16 Sep 2013 10:41:41 +0000 (14:41 +0400)
commitcbc53b28b5bc747703e0730e11e2ec25d38c57fa
tree6642cbd4586e5ceab79feaf1cfed00f54454bb09
parentb97bade0388947d59dedf0e6a0e05e182711902b
Do not copy X event in handle_one_xevent except KeyPress case.
Wnen XEvent is processed, it is unlikely to be changed except
KeyPress case, so we can avoid copying and use const pointer to
const data to make sure that an event is not changed elsewhere.
* xterm.c (handle_one_xevent): Change 2nd arg to 'const XEvent *
const' and do not create local copy except for the KeyPress event.
Use casts to avoid a few glitches.  Adjust formatting.  Add comments.
(SET_SAVED_BUTTON_EVENT): Remove and move the code to the only user.
(x_handle_net_wm_state, x_menubar_window_to_frame)
(x_detect_focus_change, construct_mouse_click, note_mouse_movement)
(x_scroll_bar_to_input_event, x_scroll_bar_expose)
(x_scroll_bar_handle_click, x_scroll_bar_note_movement):
* gtkutil.c (xg_event_is_for_menubar, xg_event_is_for_scrollbar):
* xselect.c (x_handle_property_notify, x_handle_selection_notify)
(x_handle_dnd_message):
* xsettings.c (xft_settings_event):
Use 'const XEvent * const' where appropriate.
* xterm.h, gtkutil.h, xsettngs.h: Adjust related prototypes.
src/ChangeLog
src/gtkutil.c
src/gtkutil.h
src/xselect.c
src/xsettings.c
src/xsettings.h
src/xterm.c
src/xterm.h